Skip to content

Conversation

siversm
Copy link
Member

@siversm siversm commented Sep 14, 2026

Closes #4 Issue

Sivert Smedsrud added 2 commits September 14, 2026 17:55
QueryClientProvider now wraps App in main.tsx instead of being created
inside App.tsx itself.

Issue: #4
@siversm siversm self-assigned this Sep 14, 2026
@siversm siversm requested a review from a team as a code owner September 14, 2026 16:00
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does not seem integrated in the rest of the code?

Copy link
Member

@pederoha pederoha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch moving the QueryClient.
This is a better solution.

I dont know what is up with the queryKeys.ts file?

Copy link
Member

@pederoha pederoha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great call moving TanStack client to main.tsx

@pederoha pederoha merged commit 1649599 into dev Sep 15, 2026
@pederoha pederoha deleted the 4-tanstack-query-setup branch September 15, 2026 09:43
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants